[Setting] nativewind 세팅 추가#179
Conversation
|
Warning There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure. 🔧 eslint
apps/mobile/app/(tabs)/index.tsxOops! Something went wrong! :( ESLint: 8.57.1 ESLint couldn't find the config "expo" to extend from. Please check that the name of the config is correct. The config "expo" was referenced from the config file in "/apps/mobile/.eslintrc.js". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team. 워크스루이 풀 리퀘스트는 Expo 및 웹 애플리케이션의 다양한 구성 파일과 컴포넌트에 대한 여러 변경 사항을 포함하고 있습니다. 주요 변경 사항은 Tailwind CSS 구성 업데이트, Metro 번들러 설정 조정, 오류 처리 컴포넌트 리팩토링, 그리고 몇몇 README 및 구성 파일 수정을 포함합니다. 변경 사항
시퀀스 다이어그램sequenceDiagram
participant App as 모바일 앱
participant Metro as Metro 번들러
participant Tailwind as Tailwind CSS
App->>Metro: CSS 지원 요청
Metro-->>App: CSS 구성 활성화
App->>Tailwind: 공유 구성 로드
Tailwind-->>App: 스타일 적용
시
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
🚀 Preview URLBranch: 178-setting-nativewind-세팅-추기 Preview URL: https://codeit.click?pr=179 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
apps/mobile/tailwind.config.js (1)
5-10: 스캔 경로 확장에 따른 빌드 성능 영향 주의
추가된content경로(./src/**,./components/**,../../packages/ui/src/**등 )가 많아지면 빌드 성능이 저하될 수 있습니다. 불필요한 경로가 없는지 확인해 보시길 권장드립니다.apps/mobile/metro.config.js (1)
10-15: 불필요 주석 제거 검토
monorepoPackages객체 안에 주석 처리된 경로들이 남아 있습니다. 더 이상 사용하지 않는다면 제거를 고려해 보세요.apps/web/components/common/Fallback/index.tsx (1)
15-15: 에러 복구 시 전체 쿼리 제거 가능성 점검
queryClient.removeQueries();는 모든 쿼리의 캐시를 제거하므로, 의도치 않은 데이터 재요청이 발생할 수 있습니다. 특정 쿼리만 재설정이 필요한 경우 선택적으로 제거하는 방안을 고려해 주세요.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (10)
.expo/README.md(1 hunks).expo/devices.json(1 hunks)apps/mobile/app/(tabs)/index.tsx(1 hunks)apps/mobile/app/_layout.tsx(1 hunks)apps/mobile/metro.config.js(1 hunks)apps/mobile/tailwind.config.js(1 hunks)apps/mobile/tsconfig.json(1 hunks)apps/web/components/common/ErrorResetBoundary/index.tsx(1 hunks)apps/web/components/common/Fallback/index.tsx(1 hunks)tsconfig.json(1 hunks)
✅ Files skipped from review due to trivial changes (5)
- .expo/devices.json
- tsconfig.json
- apps/mobile/app/_layout.tsx
- apps/mobile/tsconfig.json
- .expo/README.md
🔇 Additional comments (8)
apps/mobile/tailwind.config.js (2)
1-2: 공유 Tailwind 설정을 통한 확장 확인
모노레포에서 공유 설정을 가져오는 방식이 올바른지 확인해 주세요.@repo/tailwind-config/tailwind.config경로가 정상적으로 동작하는지, 의존성 문제나 모듈 충돌이 없는지 점검해 보시면 좋겠습니다.
11-11: 프리셋 병합 순서 점검
nativewind/preset뒤에sharedConfig가 추가되었습니다. 두 설정이 어떻게 병합되는지(테마 컬러의 충돌 등) 한 번 더 점검해 보세요.apps/mobile/metro.config.js (2)
8-8: CSS 활성화 옵션 확인
isCSSEnabled: true로 설정되어 Tailwind 등 CSS 기능을 사용할 수 있게 되었습니다. Expo 및 Metro 환경에서 해당 설정이 문제가 없는지 최종 확인해 주세요.
16-18: 빈 패키지 설정 유효성 점검
config.watchFolders와config.resolver.extraNodeModules에monorepoPackages가 비어 있습니다. 설정 목적으로 남겨 둔 것이라면, 주석 등으로 의도를 명확히 해주시면 좋겠습니다.apps/mobile/app/(tabs)/index.tsx (1)
15-17: Tailwind 클래스 사용 확인
className="bg-custom-black rounded-lg border-2 border-purple-500"설정이 의도한 대로 동작하는지 확인해 주세요. 공유 설정에서bg-custom-black가 정의되어 있지 않다면 예상된 스타일이 적용되지 않을 수 있습니다.apps/web/components/common/ErrorResetBoundary/index.tsx (1)
15-15: 의존성 감소 및 코드 간소화
QueryErrorResetBoundary사용을 제거하고ErrorBoundary만 적용하여 불필요한 의존성을 줄이고 코드가 간결해졌습니다. 기존 기능과 동등하게 동작하므로 특별한 문제 없이 승인 가능합니다.apps/web/components/common/Fallback/index.tsx (2)
4-4: Query Client 활용을 위한 올바른 의존성 주입
getQueryClient모듈을 명시적으로 가져와서 필요한 곳에서 활용할 수 있게 되었습니다. 올바른 위치에서 import를 처리하셨으므로 문제 없어 보입니다.
12-13: QueryClient 선언 위치 확인
오류 발생 시점부터 에러 복구 로직까지의 흐름 상queryClient를 전역이 아닌 해당 범위에서 즉시 가져오는 방식은 합리적입니다. 필요 시 다른 컴포넌트에서도 동일한 접근법을 적용할 수 있도록 고려해 보세요.
| @@ -1,8 +1,14 @@ | |||
| import sharedConfig from "@repo/tailwind-config/tailwind.config"; | |||
2 participants
🚀 작업 내용
📝 참고 사항
🖼️ 스크린샷
🚨 관련 이슈 (이슈 번호)
✅ 체크리스트
Summary by CodeRabbit
문서
.expo/README.md에 Expo 프로젝트의.expo폴더에 대한 상세 설명 추가devices.json파일 생성 및 초기 구조 설정사용자 인터페이스
HomeScreen컴포넌트의 텍스트 스타일 및 레이아웃 업데이트구성
오류 처리